Skip to content

lintrunner init can fail, don't suppress stderr #11667

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 16, 2025
Merged

Conversation

swolchok
Copy link
Contributor

Seeing it fail currently on https://github.com/pytorch/executorch/actions/runs/15647163870/job/44086495890?pr=11664 and cannot debug; let's not have this happen again.

@swolchok
Copy link
Contributor Author

swolchok commented Jun 14, 2025

Stack from ghstack (oldest at bottom):

Copy link

pytorch-bot bot commented Jun 14, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/11667

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

swolchok added a commit that referenced this pull request Jun 14, 2025
Seeing it fail currently on https://github.com/pytorch/executorch/actions/runs/15647163870/job/44086495890?pr=11664 and cannot debug; let's not have this happen again.


ghstack-source-id: 4bb8d55
ghstack-comment-id: 2972215465
Pull-Request-resolved: #11667
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 14, 2025
@swolchok swolchok requested a review from huydhn June 14, 2025 03:38
[ghstack-poisoned]
@swolchok swolchok force-pushed the gh/swolchok/460/head branch from c0356df to 267da9a Compare June 16, 2025 16:38
@swolchok swolchok added the release notes: none Do not include this in the release notes label Jun 16, 2025
@swolchok
Copy link
Contributor Author

previous green CI run except for lintrunner. lintrunner init is failing on main. merging.

@swolchok swolchok merged commit 884e901 into main Jun 16, 2025
93 of 94 checks passed
@swolchok swolchok deleted the gh/swolchok/460/head branch June 16, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants